Skip to content

css.types.basic-shape.shape - add note about unimplemented via to with change #26756

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
OnkarRuikar opened this issue May 9, 2025 · 5 comments
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS info needed This needs more information to review or act on. p3 Medium priority – Community PRs encouraged.

Comments

@OnkarRuikar
Copy link
Contributor

What type of issue is this?

Missing compatibility data

What information was incorrect, unhelpful, or incomplete?

The shape() function had a syntax change recently. The via keyword was renamed to with in the syntax.

What browsers does this problem apply to, if applicable?

No response

What did you expect to see?

It would be great if a note is added to BCD saying the function is still using via instead of with. Add the note for Chrome, Edge, Firefox. No need for Safari because it has released the change in prod.

Did you test this? If so, how?

In a local Linux dev environment.

Can you link to any release notes, bugs, pull requests, or MDN pages related to this?

Do you have anything more you want to share?

No response

MDN URL

No response

MDN metadata

No response

@OnkarRuikar OnkarRuikar added the needs triage This issue needs to be confirmed label May 9, 2025
@github-actions github-actions bot added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label May 9, 2025
@caugner
Copy link
Contributor

caugner commented May 9, 2025

Spec history:

I think it would actually make sense to add a behavioral subfeature with_keyword.

@caugner caugner added p3 Medium priority – Community PRs encouraged. and removed needs triage This issue needs to be confirmed labels May 9, 2025
@OnkarRuikar
Copy link
Contributor Author

@caugner
Copy link
Contributor

caugner commented May 10, 2025

Also, shape() is supported for clip-path property but not offset-path in Edge.

If support has landed in Chrome stable, it should also be available in Edge.

@OnkarRuikar Would you have a small test case to determine which syntax a browser supports?

@caugner caugner added the info needed This needs more information to review or act on. label May 10, 2025
@maimumah

This comment was marked as spam.

@OnkarRuikar
Copy link
Contributor Author

Would you have a small test case to determine which syntax a browser supports?

@caugner I've made one: https://jsbin.com/yiwugowugo/1/edit?html,js,output

Outputs from some browsers:

Chrome 138.0.7180.2 (Official Build) dev (64-bit)
clip-path shape(with): ✅
clip-path shape(via): ❌

offset-path shape(): ✅
offset-path shape(with): ✅
offset-path shape(via): ❌

-----------------------------------------------

Firefox 138.0.3 (64-bit)
clip-path shape(with): ❌
clip-path shape(via): ✅

offset-path shape(): ✅
offset-path shape(with): ❌
offset-path shape(via): ✅ 

-----------------------------------------------

Brave 1.78.97 Chromium: 136.0.7103.93 (Official Build) (64-bit)
clip-path shape(with): ✅
clip-path shape(via): ❌

offset-path shape(): ❌

-----------------------------------------------

Edge  136.0.3240.76 (Official build) (64-bit)
clip-path shape(with): ✅
clip-path shape(via): ❌

offset-path shape(): ❌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS info needed This needs more information to review or act on. p3 Medium priority – Community PRs encouraged.
Projects
None yet
Development

No branches or pull requests

3 participants